[DevTools] PR2: Backend Services, Console Viewer, and Resources #2879

megha-narayanan · 2025-07-02T23:34:30Z

Changes

Implements a functional DevTools UI for the Amplify Sandbox, building upon the foundation established in PR1. Key features include:

Web-based interface for managing sandbox environments
Real-time monitoring of sandbox status and deployment progress
Console log viewer with formatted logs
Resource explorer to view and manage deployed AWS resources
WebSocket-based communication between UI and backend
Enhanced sandbox lifecycle management (start, stop, delete)
Improved error handling and status reporting

The implementation uses Express with Socket.IO for the backend server and React with Cloudscape Design components for the frontend UI.

Validation

Unit tests have been included, though some files will only be further tested in later PRs

Checklist

If this PR includes a functional change to the runtime behavior of the code, I have added or updated automated test coverage for this change.
If this PR requires a change to the Project Architecture README, I have included that update in this PR.
If this PR requires a docs update, I have linked to that docs PR above.
If this PR modifies E2E tests, makes changes to resource provisioning, or makes SDK calls, I have run the PR checks with the run-e2e label set.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

…mand

changeset-bot · 2025-07-02T23:34:34Z

🦋 Changeset detected

Latest commit: f55e43c

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 7 packages

Name	Type
@aws-amplify/deployed-backend-client	Patch
@aws-amplify/platform-core	Patch
@aws-amplify/cli-core	Patch
@aws-amplify/sandbox	Minor
@aws-amplify/backend-cli	Minor
@aws-amplify/backend-deployer	Patch
@aws-amplify/ai-constructs	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Amplifiyer

Since it's quite a big PR, I've reviewed the backend pieces at a high level and left feedback. I'll take a pass at frontend components later.

Amplifiyer · 2025-07-08T17:11:21Z

package.json

+    "@aws-sdk/client-cloudformation": "^3.828.0",
+    "@aws-sdk/client-cloudwatch-logs": "^3.835.0",
    "@aws-sdk/client-cognito-identity-provider": "^3.750.0",
    "@aws-sdk/client-dynamodb": "^3.750.0",
-    "@aws-sdk/client-iam": "^3.750.0",
+    "@aws-sdk/client-iam": "^3.835.0",


Do we need aws-sdk upgrade as part of this? If not, we should decouple.

Amplifiyer · 2025-07-08T17:12:00Z

package.json

+  "dependencies": {
+    "@aws-sdk/client-lambda": "^3.835.0",
+    "cookie-signature": "1.0.6",
+    "debounce-promise": "^3.1.2",
+    "esbuild": "0.25.5",
+    "express": "^5.1.0",
+    "express-rate-limit": "^7.5.0",
+    "jszip": "^3.10.1",
+    "ws": "^8.18.2"
+  },


Do we need all these dependencies, and in the root package?

Nope. Removing

Amplifiyer · 2025-07-08T17:15:19Z

packages/cli/src/commands/sandbox/port_checker.ts

+   * Attempts to start the server on the specified port
+   * @param server The HTTP server
+   * @param port The port to use
+   * @returns A promise that resolves with the port when the server starts
+   * @throws Error if the port is already in use
+   */
+  async findAvailablePort(


The method name is not matching the documentation and what it is doing. And why are we starting a server in PortChecker class?

moving the server starting to devtools command -- this is a relic from when we tried multiple ports.

Amplifiyer · 2025-07-08T17:21:36Z

packages/cli/src/commands/sandbox/sandbox-devtools/react-app/src/components/ConsoleViewer.tsx

+const cleanAnsiCodes = (text: string): string => {
+  // This regex handles various ANSI escape sequences including colors, bold, dim, etc.
+  return text.replace(/\u001b\[\d+(;\d+)*m|\[2m|\[22m|\[1m|\[36m|\[39m/g, '');
+};


Look into stripANSI which is already used in this repo.

Have we already discussed keeping the styles offline with something like ansi-to-html?

switched to stripANSI for now-- is there a benefit to taking it offline if stripAnsi already exists in the repo?

Amplifiyer · 2025-07-08T17:23:15Z

packages/cli/src/commands/sandbox/sandbox-devtools/logging/cloudformation_format.ts

+ * @param constructPath The CDK construct path
+ * @returns A normalized construct path
+ */
+const normalizeCDKConstructPath = (constructPath: string): string => {


This seems duplicated from here

amplify-backend/packages/cli-core/src/loggers/cfn-deployment-progress/cfn_deployment_progress_logger.ts

Line 237 in 146f603

private normalizeCDKConstructPath = (constructPath: string): string => {

We should see if we can refactor cfn_deployment_progress_logger.ts such that it can be used for both CLI and DevConsole instead of duplicating the logic

pulled that function out and exported it to get rid of duplication.

Amplifiyer · 2025-07-08T17:51:06Z

packages/cli/src/commands/sandbox/sandbox-devtools/sandbox_devtools_command.ts

+      if (sandboxState === 'unknown') {
+        try {
+          // Check AWS stack state
+          const cfnClient = new CloudFormationClient();


All of these clients should be injected at runtime. Will make the unit testing much easier and simpler.

done. this does make unit testing a lot easier! I'll go ahead and make unit tests more robust in a separate commit

Amplifiyer · 2025-07-08T17:52:33Z

packages/cli/src/commands/sandbox/sandbox-devtools/sandbox_devtools_command.ts

+            await cfnClient.send(
+              new DescribeStacksCommand({ StackName: stackName }),
+            );


You are not using the response here?

we're just checking if the stack exists. but maybe using exceptions for control flow is not appropriate. Would it be better if I used ListStacksCommand or validateTemplate. Or waitUntilStackExists with a short timeout?

can't the service return undefined as well that we should check against? https://docs.aws.amazon.com/AWSJavaScriptSDK/v3/latest/Package/-aws-sdk-client-cloudformation/Interface/DescribeStacksCommandOutput/

Amplifiyer · 2025-07-08T17:54:45Z

packages/cli/src/commands/sandbox/sandbox-devtools/sandbox_devtools_command.ts

+    // Override printer.log to capture the log level
+    printer.log = function (message: string, level?: LogLevel) {
+      currentLogLevel = level ? level : LogLevel.DEBUG;
+      const result = originalLog.call(this, message, currentLogLevel);
+      currentLogLevel = null;
+      return result;
+    };
+
+    // Override printer.print (the lower-level method)
+    printer.print = function (message: string) {
+      // Call the original print method
+      originalPrint.call(this, message);
+      // Avoid double processing if this is called from printer.log
+      if (processingMessage) {
+        return;
+      }


We should avoid doing this. This creates sideeffects in other parts of code that would be very hard to debug/troubleshoot

refactored to pass in my own printer. required taking sandboxfactory out of the injected dependencies -- let me know what you think about the approach I just pushed.

Amplifiyer · 2025-07-08T17:59:41Z

packages/cli/src/commands/sandbox/sandbox-devtools/services/socket_handlers.ts

+      // Get region information
+      const cfnClient = new CloudFormationClient();
+      const regionValue = cfnClient.config.region;
+      let region = null;
+
+      try {
+        if (typeof regionValue === 'function') {
+          if (regionValue.constructor.name === 'AsyncFunction') {
+            region = await regionValue();
+          } else {
+            region = regionValue();
+          }
+        } else if (regionValue) {
+          region = String(regionValue);
+        }
+
+        // Final check to ensure region is a string
+        if (region && typeof region !== 'string') {
+          region = String(region);
+        }
+      } catch (error) {
+        printer.log('Error processing region: ' + error, LogLevel.ERROR);
+        region = null;
+      }


See if we can reuse this

amplify-backend/packages/platform-core/src/telemetry/region_fetcher.ts

Line 6 in 146f603

export class RegionFetcher {

through a DI, so we can reuse the cached results instead of finding the region every time.

yep, this also makes testing easy. Adding it.

Amplifiyer · 2025-07-08T18:00:47Z

packages/cli/src/commands/sandbox/sandbox-devtools/services/socket_handlers.ts

+/**
+ * Service for handling socket events
+ */
+export class SocketHandlerService {


This class also needs required unit test coverage

yep, I hadn't put it in this PR. updated and added.

…nd added to dependencies

svidgen

Can't say I did a full review here — there's a lot here. 🎉 But, I left a handful of comments for you for things that jumped out at me.

My other big concern across the board is testing. Given that this is destined for a feature branch, and given that it's already huge, I could be convinced to see a separate PR focused on e2e testing. Some lower level unit/interaction testing would be good too. But, long term, before I'd put this in front of customers, I'd want to know I "can't" break it. And, I think e2e's are how I gain that confidence.

svidgen · 2025-07-08T18:30:14Z

packages/cli/src/commands/sandbox/sandbox-devtools/react-app/src/components/ConsoleViewer.tsx

+const cleanAnsiCodes = (text: string): string => {
+  // This regex handles various ANSI escape sequences including colors, bold, dim, etc.
+  return text.replace(/\u001b\[\d+(;\d+)*m|\[2m|\[22m|\[1m|\[36m|\[39m/g, '');
+};


Have we already discussed keeping the styles offline with something like ansi-to-html?

svidgen · 2025-07-08T21:35:04Z

packages/cli/src/commands/sandbox/sandbox-devtools/logging/cloudformation_format.ts

+/**
+ * Clean ANSI escape codes from text
+ * @param text The text to clean
+ * @returns The cleaned text
+ */
+export const cleanAnsiCodes = (text: string): string => {
+  // Split the regex into parts to avoid control characters
+  const ansiEscapeCodesPattern = new RegExp(
+    [
+      // ESC [ n ; n ; ... m
+      String.fromCharCode(27) + '\\[\\d+(?:;\\d+)*m',
+      // Other common ANSI sequences
+      '\\[2m',
+      '\\[22m',
+      '\\[1m',
+      '\\[36m',
+      '\\[39m',
+    ].join('|'),
+    'g',
+  );
+
+  return text.replace(ansiEscapeCodesPattern, '');
+};


What's functionally different about this cleaner and the one below?

already resolved in commit 87fa132 (duplicate from trimming down PR)

svidgen · 2025-07-08T21:38:11Z

packages/cli/src/commands/sandbox/sandbox-devtools/logging/cloudformation_format.ts

+/**
+ * Extracts a friendly name from a nested stack logical ID
+ * @param stackName The stack name to process
+ * @returns A friendly name or undefined if no match
+ */
+const getFriendlyNameFromNestedStackName = (
+  stackName: string,
+): string | undefined => {
+  const parts = stackName.split('-');
+
+  if (parts && parts.length === 7 && parts[3] === 'sandbox') {
+    return parts[5].slice(0, -10) + ' stack';
+  } else if (parts && parts.length === 5 && parts[3] === 'sandbox') {
+    return 'root stack';
+  }
+
+  return undefined;
+};


A good habit when baking arcane splits and magic numbers into a function like this is to show an example or two of the before/after in the docstring. If there is a good name you can give to the magic constants here, give them names. Else, it's helpful to indicate where they come from in the docstring examples.

In this case, the naming patterns might be completely obvious to others in the codebase. But, if someone new (it's always Day 1, right?) jumps in at this point, examples will help.

Leaving this comment on this particularly arcane looking set of manipulations. But, please apply anywhere you feel arcane incantations are being performed.

got it. actually figured out how to actually get non-null metadata which eliminates the need for some of these functions. but examples have been added in the remaining functions where I felt it was needed.

svidgen · 2025-07-08T21:40:22Z

packages/cli/src/commands/sandbox/sandbox-devtools/logging/cloudformation_format.ts

+ * @param metadata.constructPath Optional construct path from CDK metadata
+ * @returns A user-friendly name for the resource
+ */
+export const createFriendlyName = (


Borderline arcane incantations herein. Partially cp-ing a comment from below. Examples in docstrings will help future maintainers.

Thank-you! Probably goes without saying, but explicit test coverage would also be ideal. Unless I'm overlooking it, just a small set of examples like you have here in a test loop would be good.

svidgen · 2025-07-08T21:48:30Z

packages/cli/src/commands/sandbox/sandbox-devtools/logging/cloudformation_format.ts

+    cleanedMessage.includes('_IN_PROGRESS') ||
+    cleanedMessage.includes('CREATE_') ||
+    cleanedMessage.includes('DELETE_') ||
+    cleanedMessage.includes('UPDATE_') ||
+    cleanedMessage.includes('Deployment in progress') ||
+    cleanedMessage.includes('COMPLETE') ||
+    cleanedMessage.includes('FAILED') ||


Wouldn't mind seeing something like this if it speaks to you:

[ '_IN_PROGRESS', 'CREATE_', 'DELETE_', 'UPDATE_', 'Deployment in progress', 'COMPLETE', 'FAILED' ].some(pattern => cleanedMessage.includes(pattern))

But, with room for opinion here, so adopt only if it really speaks to you.

svidgen · 2025-07-08T22:06:28Z

packages/cli/src/commands/sandbox/sandbox-devtools/services/shutdown_service.ts

+
+    // Exit the process if requested
+    if (exitProcess) {
+      // Short delay to allow messages to be sent


What messages? Why is 500 ms the right amount of time to wait for them to send?

I was sending a shutdown log but I guess that isn't super helpful because the process is, well, shutting down. If we want to log, we can always log on socket disconnect. Getting rid of the log and delay. This delay was also for the server close, which is async but doesn't return a promise. resolving that properly with a promise wrapper to make sure server closing is finished before we end the process.

svidgen · 2025-07-08T22:09:31Z

packages/cli/src/commands/sandbox/sandbox-devtools/services/socket_handlers.ts

+    private sandbox: Sandbox,
+    private getSandboxState: () => Promise<string>,
+    private backendId: { name: string },
+    private shutdownService: import('./shutdown_service.js').ShutdownService,


Why import() ShutdownService as a function here instead of letting it hang out with its other import-ant friends?

Moving it to the top with the other imports!

svidgen · 2025-07-08T22:15:31Z

packages/cli/src/commands/sandbox/sandbox-devtools/react-app/src/hooks/useResourceManager.ts

+    socket.on('savedResources', handleSavedResources);
+    socket.on('deployedBackendResources', handleDeployedBackendResources);
+    socket.on('customFriendlyNames', handleCustomFriendlyNames);
+    socket.on('customFriendlyNameUpdated', handleCustomFriendlyNameUpdated);
+    socket.on('customFriendlyNameRemoved', handleCustomFriendlyNameRemoved);
+    socket.on('error', handleError);


Just picking a random spot to leave this comment: Would be really good to have constants imported from a central spot (at least per group) to match on instead of duplicated strings all over. In theory, a typo in one of those strings could be caught by a good suite of tests as well, but caught much sooner if caught statically.

(Bonus points if you include docstrings to indicate what the constants mean.)

Most IDEs can also help track down usage of a variable/constant more definitively than searching for a string literal.

done. put constants in a shared location that frontend and backend can both access.

svidgen · 2025-07-08T22:19:43Z

packages/cli/src/commands/sandbox/sandbox-devtools/services/resource_service.ts

+            if (regionValue.constructor.name === 'AsyncFunction') {
+              region = await regionValue();
+            } else {
+              region = regionValue();
+            }


Q for the team: Do we have lint rules in place that would prevent this from just being await regionValue() either way? If not, there's usually not much reason to check whether a thing is async. If it can be async, just await it. (Right?)

resolved by eliminating this code by using RegionFetcher

svidgen · 2025-07-08T22:23:42Z

packages/cli/src/commands/sandbox/sandbox-devtools/resource_console_functions.ts

+    // case 'AWS::DynamoDB::Table': // COMMENT OUT (UNTESTED)
+    //   // Check if physicalId is an ARN and extract table name if needed
+    //   const dynamoTableName = physicalId.includes(':table/')
+    //     ? physicalId.split(':table/')[1]
+    //     : physicalId;
+    //   return `${baseUrl}/dynamodb/home?region=${region}#/tables/view/${encodeURIComponent(dynamoTableName)}`;


You mean not even manually tested? The others aren't covered by unit or integ tests yet, are they?

yep, I did manual testing now though so I've added these back in.

…eploymentProgressLogger and createFriendlyName. maintains backward compatibility

- Introduced SocketClientService to handle socket connections and events. - Created SandboxClientService and ResourceClientService for sandbox and resource-specific socket communication. - Updated App component to utilize context for socket services. - Refactored useResourceManager hook to use ResourceClientService. - Updated ResourceConsole and Header components to use new service structure.

Amplifiyer · 2025-07-16T15:19:05Z

packages/cli/src/commands/sandbox/sandbox_event_handler_factory.ts

+      resourceConfigChanged: [
+        async () => {
+          try {


I can't find where this event is being emitted by the sandbox

Yep, its extraneous now that it been replaced with successfulDeployment/failedDeployment for better error handling. removed in PR3 but forgot to propogate back to PR2. Doing that now.

Amplifiyer · 2025-07-16T15:21:07Z

packages/sandbox/src/sandbox.ts

+  /**
+   * Gets the current state of the sandbox
+   * @returns The current state: 'running', 'stopped', 'deploying', 'deleting', 'nonexistent', or 'unknown'
+   */
+  getState: () => SandboxStatus;


This state management needs more unit test coverage.

Absolutely. In writing my tests I also discovered a tiny edge case error in my state management -- fixed that and added tests.

Amplifiyer · 2025-07-16T16:06:23Z

packages/deployed-backend-client/src/deployed-backend-client/deployed_resources_enumerator.ts

+        metadata:
+          templateMetadata[stackResourceSummary.LogicalResourceId || ''],


Should this be

Suggested change

metadata:

templateMetadata[stackResourceSummary.LogicalResourceId || ''],

metadata:

templateMetadata[stackResourceSummary.LogicalResourceId] || '',

EDIT: okay, turns out I actually misunderstood this. The reason I had my original code was because in case LogicalResourceId is undefined (which it can be), then we just look for the the metadata of ''. Though I imagine this is not the best way to handle that error. The metadata property should be of type{ constructPath?: string } | undefined, not string. I have now re-updated this to the following line, which is more intentional about handling the undefined case - instead of falling back to an empty string key (which likely doesn't exist in the metadata, but maybe in an error case, COULD), it explicitly returns undefined. It avoids a potential issue where templateMetadata[''] might actually contain something unexpected. stackResourceSummary.LogicalResourceId ? templateMetadata[stackResourceSummary.LogicalResourceId] : undefined,

…eployed_resources_enumerator.ts Co-authored-by: Amplifiyer <51211245+Amplifiyer@users.noreply.github.com>

…r SandboxOptions

…yanan/amplify-backend into pr-2-backend-services Merge remote changes

packages/cli/src/commands/sandbox/sandbox-devtools/react-app/src/components/Header.tsx

Amplifiyer

Great job 🚀

Amplifiyer · 2025-07-18T19:12:40Z

packages/cli/src/commands/sandbox/sandbox-devtools/react-app/src/components/ConsoleViewer.tsx

+      case 'debug':
+        return 'pending';
+      default:
+        return 'info';


why is debug level a pending status and default info?

This was just for the UI styling of the little icon that goes next to the level. This is the type:
export declare namespace StatusIndicatorProps { type Type = 'error' | 'warning' | 'success' | 'info' | 'stopped' | 'pending' | 'in-progress' | 'loading'; type Color = 'blue' | 'grey' | 'green' | 'red' | 'yellow'; } I just thought that the icon for pending fit debug best, and info was most versatile. Is there a choice for either that you think would be better?

got it. Maybe the name is slightly confusing. Perhaps getStatusIndicatorIconForLogLevel (verbose but self documenting)

packages/cli/src/commands/sandbox/sandbox-devtools/react-app/src/components/ConsoleViewer.tsx

Amplifiyer · 2025-07-18T19:14:41Z

packages/cli/src/commands/sandbox/sandbox-devtools/react-app/src/components/Header.tsx

+    console.log(
+      `[CLIENT] Header: sandboxStatus prop changed to ${sandboxStatus}`,
+    );
+


do we need these log statements?

nope, removing.

Amplifiyer · 2025-07-18T19:19:38Z

packages/cli/src/commands/sandbox/sandbox-devtools/react-app/src/components/Header.tsx

+        if (statusCheckTimeout === 0) {
+          return (
+            <StatusIndicator type="pending">
+              Checking Sandbox Status
+            </StatusIndicator>
+          );
+        } else if (statusCheckTimeout === 1) {
+          return (
+            <StatusIndicator type="pending">
+              Still checking status...
+            </StatusIndicator>
+          );
+        } else {
+          return (
+            <SpaceBetween direction="horizontal" size="xs">
+              <Spinner size="normal" />
+              <StatusIndicator type="pending">
+                Status check taking longer than expected
+              </StatusIndicator>
+            </SpaceBetween>
+          );
+        }


We don't need this level of messaging. It's either "Waiting for status..." or "${Status}" to keep it simple.

Amplifiyer · 2025-07-18T19:33:41Z

packages/cli/src/commands/sandbox/sandbox-devtools/logging/cloudformation_format.test.ts

@@ -0,0 +1,116 @@
+/* eslint-disable spellcheck/spell-checker */


In one of future PR, we should remove all file level eslint suppression.

yep, will do.

rtpascual

Overall LGTM 🚀

…rc/components/ConsoleViewer.tsx Co-authored-by: Amplifiyer <51211245+Amplifiyer@users.noreply.github.com>

rtpascual · 2025-07-18T21:43:19Z

packages/cli/src/commands/sandbox/sandbox-devtools/react-app/src/components/Header.tsx

  useEffect(() => {
-    console.log(
-      `[CLIENT] Header: sandboxStatus prop changed to ${sandboxStatus}`,
-    );
-
-    // Always reset loading state when status changes, regardless of the new state
-    console.log(
-      `[CLIENT] Header: Resetting isLoading to false due to status change: ${sandboxStatus}`,
-    );
    setIsLoading(false);
-
-    // If status is still unknown after a delay, increment the timeout counter
-    // to show a more informative message
-    if (sandboxStatus === 'unknown') {
-      const timer = setTimeout(() => {
-        setStatusCheckTimeout((prev) => prev + 1);
-      }, 3000);
-      return () => clearTimeout(timer);
-    } else {
-      // Reset timeout counter when we get a definitive status
-      setStatusCheckTimeout(0);
-    }
  }, [sandboxStatus]);


This can be in a follow up PR but I believe this useEffect can be removed since this component rerenders when sandboxStatus changes (since it is a prop) and you already initialize the state as false in line 35.

Megha Narayanan added 8 commits June 25, 2025 16:32

PR1: Dependencies, Basic React App Facade, and Barebones Devtools Com…

e994339

…mand

made timeout longer AND temporary workaround for license issues

02a7289

PR2: Basic Sandbox Manager and Resource Console PRELIM

776fd8a

Fixing Lint and Tests

7bd6b1b

fixed lint issues and tests

093c130

updating for structure changes, printer

c9d0917

merging with pr-1 as base

ce447ae

update API

8ff3baf

Megha Narayanan added 3 commits July 2, 2025 16:51

updating package_lock (based on pr_1)

3085faf

update default_packages

d9d10c9

added changeset

9955896

megha-narayanan marked this pull request as ready for review July 3, 2025 00:13

megha-narayanan requested review from a team as code owners July 3, 2025 00:13

Megha Narayanan added 2 commits July 2, 2025 17:29

fix lint

1a0849a

reconcile status types

ce3a461

Amplifiyer reviewed Jul 8, 2025

View reviewed changes

Megha Narayanan added 4 commits July 8, 2025 12:44

removed profile from start sandbox modal as it can't do anything

134e83a

fixed loading of start sandbox when modal dismissed

52d92e0

removed aws-sdk upgrade, removed dependencies in root package

77b7f60

got rid of cleanAnsiCodes (and duplicate), added stripAnsi instead, a…

87fa132

…nd added to dependencies

svidgen reviewed Jul 8, 2025

View reviewed changes

Megha Narayanan added 7 commits July 8, 2025 15:40

moved server starting from portchecker to devtools command

740271c

fix lint

68c9ab7

pull out normalizeCDKConstructPath utility and refactor usage in CfnD…

73f32e0

…eploymentProgressLogger and createFriendlyName. maintains backward compatibility

fix api for cli-core (added normalizeCDK constructPath)

dc33887

make AWS console links visible, remove unused log note

11c9684

Update sandbox state management, add initialization error handling

fa14390

combined useEffects in App.tsx

76b21a3

Amplifiyer reviewed Jul 16, 2025

View reviewed changes

Megha Narayanan and others added 8 commits July 16, 2025 09:08

remove resourceConfigChanged

64c0c4d

added error for failed emit

109f154

added unit tests for state management

b8fc07b

Update packages/deployed-backend-client/src/deployed-backend-client/d…

c9d2288

…eployed_resources_enumerator.ts Co-authored-by: Amplifiyer <51211245+Amplifiyer@users.noreply.github.com>

get rid of unused fields in StartSandboxModal, fix internal typing fo…

9416275

…r SandboxOptions

Merge branch 'pr-2-backend-services' of https://github.com/megha-nara…

9b8693a

…yanan/amplify-backend into pr-2-backend-services Merge remote changes

expand issue comment

8db171f

fix error in deployed_resources_enumerator

b271bad

rtpascual reviewed Jul 16, 2025

View reviewed changes

packages/cli/src/commands/sandbox/sandbox-devtools/react-app/src/components/Header.tsx Outdated Show resolved Hide resolved

Megha Narayanan added 7 commits July 16, 2025 13:26

throw error on null socket

f569654

inject logger factory as dependency

1004c78

header status text fix

5786485

fix lint

4861a0d

fixed unsubscribe naming.

1c6d9c0

handle undefined response on describestacks command

e5d1da0

rate limiting comment

422058c

Amplifiyer previously approved these changes Jul 18, 2025

View reviewed changes

rtpascual previously approved these changes Jul 18, 2025

View reviewed changes

header minor fixes

e3c1c9b

megha-narayanan dismissed stale reviews from rtpascual and Amplifiyer via e3c1c9b July 18, 2025 20:27

megha-narayanan and others added 2 commits July 18, 2025 13:27

Update packages/cli/src/commands/sandbox/sandbox-devtools/react-app/s…

aaa7a5a

…rc/components/ConsoleViewer.tsx Co-authored-by: Amplifiyer <51211245+Amplifiyer@users.noreply.github.com>

change fn name

f55e43c

rtpascual approved these changes Jul 18, 2025

View reviewed changes

Amplifiyer approved these changes Jul 18, 2025

View reviewed changes

megha-narayanan merged commit 28879e3 into aws-amplify:feature/dev-tools Jul 18, 2025
50 of 52 checks passed

		metadata:
		templateMetadata[stackResourceSummary.LogicalResourceId \|\| ''],

		@@ -0,0 +1,116 @@
		/* eslint-disable spellcheck/spell-checker */

[DevTools] PR2: Backend Services, Console Viewer, and Resources #2879

[DevTools] PR2: Backend Services, Console Viewer, and Resources #2879

Uh oh!

Conversation

megha-narayanan commented Jul 2, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Changes

Validation

Checklist

Uh oh!

changeset-bot bot commented Jul 2, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🦋 Changeset detected

Uh oh!

Amplifiyer left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

svidgen left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

megha-narayanan commented Jul 2, 2025 •

edited

Loading

changeset-bot bot commented Jul 2, 2025 •

edited

Loading

megha-narayanan Jul 9, 2025 •

edited

Loading